Skip to content

feat(geospatial): add alchemist-geospatial module for real-world climate data - #5556

Open
Wanes01 wants to merge 134 commits into
AlchemistSimulator:masterfrom
Wanes01:alchemist-geospatial
Open

feat(geospatial): add alchemist-geospatial module for real-world climate data#5556
Wanes01 wants to merge 134 commits into
AlchemistSimulator:masterfrom
Wanes01:alchemist-geospatial

Conversation

@Wanes01

@Wanes01 Wanes01 commented Aug 17, 2026

Copy link
Copy Markdown

No description provided.

@codecov

codecov Bot commented Aug 17, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.53%. Comparing base (763f808) to head (8a18c40).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff            @@
##             master    #5556   +/-   ##
=========================================
  Coverage     61.53%   61.53%           
  Complexity       14       14           
=========================================
  Files             2        2           
  Lines            78       78           
  Branches          4        4           
=========================================
  Hits             48       48           
  Misses           24       24           
  Partials          6        6

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mergify

mergify Bot commented Aug 18, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@Wanes01
Wanes01 force-pushed the alchemist-geospatial branch from 4e73252 to 6da2fa4 Compare August 18, 2026 10:34
DanySK
DanySK reviewed Aug 18, 2026
View reviewed changes

@DanySK DanySK left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are several critical issues that must be solved before I go ahead with the review. See comments.

Comment thread alchemist-full/build.gradle.kts Outdated
Comment thread ...emist-geospatial/src/main/kotlin/it/unibo/alchemist/boundary/acquisition/utility/Archives.kt Outdated
Comment thread ...emist-geospatial/src/main/kotlin/it/unibo/alchemist/boundary/acquisition/utility/Archives.kt Outdated
Comment thread alchemist-maps/build.gradle.kts Outdated
Comment thread build.gradle.kts Outdated
@DanySK
DanySK force-pushed the alchemist-geospatial branch 11 times, most recently from fbd2faa to 8337419 Compare August 20, 2026 20:26
@mergify

mergify Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Hi @Wanes01! 👋
This pull request has conflicts 😖
Could you fix it? 🔧
Thank you! 🙏

1 similar comment
@mergify

mergify Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Hi @Wanes01! 👋
This pull request has conflicts 😖
Could you fix it? 🔧
Thank you! 🙏

@Wanes01
chore(geospatial): add basic gradle configuration
4b44889
@Wanes01
feat(geospatial): add RasterGrid interface and ArrayRasterGrid implem…
47fe24c 
…entation
@Wanes01
feat(geospatial): add TimedGrid and CdmTimedGrid implementation
4b34f0a
@Wanes01
chore(geospatial): update cdm dependency version to latest (5.9.1)
fdd71d9
@Wanes01
test(geospatial): add tests for ArrayRasterGrid and TestCdmTimedGrid
a3beade
@Wanes01
feat(geospatial): add Axes helpers for index finding and their tests
5573889
@Wanes01
feat(geospatial): add SpatialInterpolationStrategy and an enum with b…
5d31a8d 
…uil-in implementation
@Wanes01
test(geospatial): add test for SpatialInterpolation enum
15103e2
@Wanes01
feat(geospatial): add SpatialExtrapolationStrategy interface and buil…
be7ef13 
…t-in strategies
@Wanes01
refactor(geospatial): modified file name from SpatialExtrapolation to…
071fbd2 
… SpatialExtrapolationStrategy
@Wanes01
test(geospatial): add tests for SpatialExtrapolation built-in strategies
70e3259
Wanes01 and others added 27 commits September 17, 2026 18:59
@Wanes01
fix(geospatial): fix EagerGridSnaphots failing on GRIB files
59c3930
@DanySK @Wanes01
build: use a custom version of CDM that can be fetched from Maven Cen…
558842b 
…tral
@Wanes01
test(geospatial): tests for GRIB file reading
3a542f3
@Wanes01
build(maps): stop using a protobuf fixed version
7a7e870
@Wanes01
build(full): stop using a protobuf fixed version
e7b47bd
@Wanes01
fix(geospatial): swap sax parser for NecdfDatasets.openDataset
a4abe71
@Wanes01
fix(full): merge plugin.properties if there are multiple entries
084d40c
@DanySK @Wanes01
build: add AppendingTransformer for merging plugin.properties in Shad…
dd47c61 
…owJar
@Wanes01
docs(geospatial): align module doc to its latest changes; various doc…
9b2e6a7 
… fixes
@Wanes01
test(geospatial): add test to detect compatibility issues between OSM…
0036ae8 
…Environment and DoubleCopernicusLayer
@Wanes01
build(full): remove the duplicate AppendingTransformer under the Shad…
25987a5 
…owJar task
@Wanes01
refactor(geospatial): make the filesystem cache manager provider-agno…
37eed00 
…stic
@Wanes01
refactor(geospatial): rename CacheKey's method from 'toFileName' to '…
8ea592e 
…toDirectoryName'
@Wanes01
refactor(geospatial): move endpoint from CopernicusDataStoreProvider …
d4b631f 
…to CopernicusRequest; refactor provider tests
@Wanes01
refactor(geospatial): move costructor defaults to DoubleCopernicusLay…
2a6cae7 
…er, CdsApiRc and FileSystemCacheManager
@Wanes01
refactor(geospatial): move spatial inclusion check into RasterGrid
6495e03
@Wanes01
feat(geospatial): fetch each cache entry at most once
d8c472a
@Wanes01
refactor(geospatial): turn toSimulationTime into an extension function
d0ad929
@Wanes01
refactor(geospatial): move terminal and running statuses in two sets
0f63f59
@Wanes01
refactor(geospatial): reorder arguments in CopernicusLayer
fdaf580
@Wanes01
refactor(geospatial): replace java Duration with kotlin Duration in C…
85c025d 
…opernicusDataStoreProvider
@Wanes01
feat(geospatial): add timeout argument to the third constructor of Co…
57c6d2e 
…pernicusLayer
@Wanes01
refactor(geospatial): remove redundant if branches in CopernicusLayer…
1730b6d 
….getValue
@Wanes01
test(geospatial): test FileSystemCacheManager behavior on multiple pr…
848e0d9 
…ocesses fetching the same assets
@Wanes01
refactor(geospatial): remove code duplication in tests
fde7aae
@Wanes01
refactor(geospatial): make CacheManager a functional interface
b9f8052
@Wanes01
refactor(geospatial): reduce rasterGridContract's cognitive complexity
8a18c40
@Wanes01
Wanes01 force-pushed the alchemist-geospatial branch from 8e54f3b to 8a18c40 Compare September 17, 2026 17:08
@sonarqubecloud

sonarqubecloud Bot commented Sep 17, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mergify

mergify Bot commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Hi @Wanes01! 👋
This pull request has conflicts 😖
Could you fix it? 🔧
Thank you! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DanySK DanySK DanySK left review comments
+1 more reviewer
@github-advanced-security github-advanced-security[bot] github-advanced-security[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

auto-update-rebase

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Wanes01 @DanySK @github-advanced-security